Skip to content

Conversation

@AdvaitTahilyani
Copy link
Collaborator

LiveViz Implementation for Charm4Py.

@AdvaitTahilyani AdvaitTahilyani requested a review from ritvikrao June 2, 2025 22:42
@ritvikrao
Copy link
Collaborator

ritvikrao commented Jun 3, 2025

Tried it on my Mac and it works quite well! I have a couple of requests:

  1. Can you create the liveViz particle exercise as a separate file within a new 'liveViz' folder? Also, restore the existing ccs_server example
  2. Can you add a README explaining how to run this, and can you include a screenshot of what liveViz would look like if run correctly?
  3. Can you update the documentation with a new page that briefly describes liveViz and how to run it? As a side note, the goal of liveViz is to visually show the effects of migration and load balancing, so you should try adding command-line options like +balancer GreedyLB and +balancer RefineLB to see the effects of those load balancing strategies on the particles.

@AdvaitTahilyani
Copy link
Collaborator Author

Tried it on my Mac and it works quite well! I have a couple of requests:

  1. Can you create the liveViz particle exercise as a separate file within a new 'liveViz' folder? Also, restore the existing ccs_server example
  2. Can you add a README explaining how to run this, and can you include a screenshot of what liveViz would look like if run correctly?
  3. Can you update the documentation with a new page that briefly describes liveViz and how to run it? As a side note, the goal of liveViz is to visually show the effects of migration and load balancing, so you should try adding command-line options like +balancer GreedyLB and +balancer RefineLB to see the effects of those load balancing strategies on the particles.

Hopefully made all the changes for an initial merge. Will fiddle around with the balancers for the example. I also ended up implementing a poll mode with a pretty straightforward queue system, so do let me know if everything looks ok or whether there's something I need to change.

@AdvaitTahilyani AdvaitTahilyani merged commit 2b056c0 into main Jun 4, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants